Leak fixes in v900

Compile fix for Ubuntu. Fix valgrind warning from scribble code.

BAM, NEW_STRINGS is now the default.
(Now off to deal with the inevitable leaks I just introduced...)

Let zconf pick up our config.h.  Regenerate dependencies.

Fix zlib, warning tweaks.

Update to zlib 1.2.8.

Update to shapelib 1.3.0

"fix" use after free in tef.

Fix (well, paper over...) heap-use-after-free in psitrex.

Introduce leaks (probably) in igc and google for now, just to get
code building and passing in NEW_STRINGS.

Fix a test case error in gdb caused by excessive bogusness.  I have No Idea
why extra_data is involved in this path.

This is a groady submit.  This is where I take all of the things that were
tacky or sleazy related to character encoding and basically apply pain killer
to them.  Most of the tests were clearly made up and were just
institutionalizing what we did instead of thinking about what was right.
Remove the '-c' flag.

A further beatdown on binary option handling in the GUI.  I think it
finally works sensibly now.

Mechanical rename of class member variables in the GUI.  No real changes.

Ignore talker identification in nmea reader.

Rework the way boolean options are handled in the GUI. They're now
emitted only if they're not the default.

Rename member variables to use foo_ convention.

fix some encoding issues with openoffice.style.
regenerate internal_styles.cc, deleteing previously deprecated styles.

fix declaration of phony targets.
a failure example was "make;touch gui;make gui", which would fail to execute the gui recipe.

fix sign issue with Qt > 5.0 in gui.

Remove extra -I when building on Mac.  No reason to search that directory
a zillion times when only one file needs mac/libusb/

Steve teaches configure/make about Mac Qt 5.2.0s use of frameworks.

Quiet sed regular expression errors in dop_filter.test

Fix most of our problems in GDB which stupidly uses Latin1 some of the
time and UTF8 some of the time.

There's still a bug in the GDB writer, I think.

ignrando: drop straydebugging.
xmlgeneric: pass qstrings all the way through
cet_util: don't molest rte_name, rte_desc with potential double encodings.

Fix BCR in NEW_STRINGS.

Deprecate Kartex, a program out of business for years and whose
only appearance on our mailing list was from the contributor.

Fix an1 for NEW_STRINGS.

Fixes for stmwpp.cc in NEW_STRINGS.

Another GDB checking for NEW_STRINGS.  This format is still a landmine.

Roll back swmwpp.  I'm out of soul to fight with it further.

Allow GPX reader to terminate if it detects Invalid state.

Improvements in stmwpp for NEW_STRINGS.

Plow through the remainder of our REALLY_MINIMAL targets with NEW_STRING.

Bring IGC to NEW_STRINGS.

Minimally invasive set of NEW_STRING changes for mmo.

Beter fxi in gdb

Leak fix in gdb.

Leak fixes for gdb.

Bring in NEW_STRINGS for gdb, saroute, igc, garmin, delbin, and an1.
Not because they actually work (in fact, they're all believed problematic)
but just to get them committed.

Fix various leaks introduced in the non-NEW_STRING case when refactoring
surrounding code to be more Qt-ish.

Rollback two formats where I introduced a regression in the non-NEW_STRINGS
case
svn merge -r4639:4637 stmwpp.cc mmo.cc

Misc minor fixes for the !NEW_STRINGS case.

Bring in csv_util.cc from NEW_STRING.

Commit the (hopefully) non-contentuous NEW_STRING changes.  This is the
batch that's largely mechanical and for formats, at least mostly passing
testo.  It's all still disabled by default and much of it is pretty dicey,
but this patch has been growing in my tree and I wanted to get it committed.

Later, when we take out the NEW_STRING stuff, I'll refactor the stuff around
these changes which were often done to preserve fidelity to the original C
versions even when there is a better way now.

Begin landing NEW_STRINGS work: common utilities.

Take another run at NMN4.

Pad internal buffer to appease -fsanitize=address.  format_garmin_xt_decrypt_trk_blk is definitely going one byte too deep on that final byte and this is probably going to come back to byte me.

Revert nmn4 change for now.  I'll deal with it later.

Make Navigon reader use QString more effectively.

Scratch in waypoint proximity in Garmin GPX writer.

Warning cleanpus from Conrad.

Fix leak I introdued in new "obvious" code yesterday.  Brace ourselves for
more of these soon...

Update .pro file from Gerhard.

Bring in some of the straight-forward pieces from NEW_STRINGS work.

A completely mechanical search and replace to swap the first argument
for an xml_generic callback with a typdef that can be swapped at compile
time.

Yes, this is a hint of things to come to reduce future bulky merges.

Compilation fixes.

Land QString changes that don't require (blatant) conditional compilation
that are really mechanical.

Make unicsv a little more QString ready internally.

Bring forward uncontested change from NEW_STRINGS work to fix non-NEW_STRINGS build break.

Commit the least contestable part of a CL I have in progress. Abandon the idea of a stepping stone "Strings" class.  Just go straight to QString, currently behind a #define.

Tweak testo to use more common stdout capture.

Guilhem Bonnefile tweaks (old) Magellan Explorist PMGNCMD,END issues.

Warning fixes from Conrad.

add utf-8 bom check and well-formed xml check to testo.

Warning cleanups from Conrad.

change illegal xml character stripping from QRegExp/QRegularExpression to QTextCodec for efficiency.  Thanks Conrad.

Preserve note for future possibilities of strip_html.

More pulling at removing C strings internally: unicsv_print_str and strenquote.

A victim of this is the seemingly unused sportsim format. It was doing
stupid things in the format definition with high byte characters and
unclear encodings.  After a few hours trying to save it, I just chopped it.

Replace guts of manual trimmed_strdup in tef reader.  Reduce type thrash.

Remove open-coded Vector in unicsv.  Make better use of QString internally.

Commit 'ptesto', a script for running valgrinds in parallel.  Really crude,
but still kind of handy.

Correct leak in bcr that I just introduced when I partially rolled back a failed experiment in making S1 and S2 into QStrings.

A bulky, but rather brainless pass at treating the important strings in
struct waypoint as a class instead of C string.  Of course, that struct
has one member - a C string - but this starts to hightlight the places
that are going to be problem areas for us.

With NEW_STRINGS turned on, we basically run, but suffer from a double
encoding of non-ascii strings.  Every format seems to be affected, so
it's probably in CET somewhere.  This isn't a QString thing in any obvious
way, as the underlying data types are still actually C Strings.

From here, additional passes can be made, either turning the individual
string pointer conversions off and whacking on a few files at a time or
going full money and making String a QString.

It does highlight the need for better vararg-style functions like warning,
fatal, and our debugging printfs to to know about stream operators.

Fixes for optionsdlg: only use ranges when we HAVE ranges (and they're not equal).  Let help link directly to the correct page.
Whitespace cleanup.

catch a straggler up with qt include renaming.

add xmlstreamwriter methods analogous to QXmlStreamWriter that strip illegal characters.

Get the gui ready for Qt5 while maintaining support for Qt4.

catch gui translation files up with recent source renaming and UpgradeCheck class work.

Minor internal changes to make new_strings land better.

Fix mem leak I introduced earlier tonight (this morning?) in ozi.

Pull out a little more char* from our core infrastructure.  Add shims to turn down case_ignore_strmp and case_ignore_strncmp().
Tweak reference files to deal with more correct(?) ordering in edge cases.
Start sketching a shim layer for shortname, description, and text, but increasingly I'm of the opinion that our only options are either do it all in one submit (eeek!) or to add additional members of struct waypt that are allocated/copied at waypt_add time so we can then at least separate readers and writers.  Instinct tells me the latter will result in harsh punishment as we'll have members that are out of sync and it will be difficult to flush them out.  So substantial parts of this CL may get tossed.

Steve replaces use of deprecated QHttp with QNetworkAccessManager.

Peter Popovec tweaks gtrnctr to fix sometimes missing lap info on write.

Remove explicit xstrdup(CSTR... in favor of helper.

Style Cleanup from last commit.

Dirk Stitchling adds support for Navin/ZNEX miniHomer 2.8 with 10Hz time.

Fix (probably) memory leak in xmltag converion.

Move remaining char*'s in xmltag to QStrings.  Tweak various callers and
copy ctors.   vtesto on CentOS 6.4 seems no worse for the wear.

Remove some read-only struct members in xml_tag.  Passes vtesto.

Fix another new/xfree mismatch.

Correct allocation in xml_tag pseudo copy constructor.

A very conservative move of xml_tag into a class of its own with
headers of its own. Tweak xmalloc/xfree to new/delete.   Regenerate deps.

fix waypoint counting errors in tpo and igc readers.
add a filter to check the counts against the lists.

fix memory leaks in cup, easygps, mapsend, osm, pcx and skyforce exposed by test-all, valgrind and patience.

fix issue with naviguide reader double counting route waypts.

fix bug in garmin_txt reader that could result in uninitialised value(s) being used when the input does not contain "Date Modified" data.

fix iblue styles, writing was causing usage of uninitialised byte(s).
fix iblue style prologue to match written fields, text taken from 2010-09-13 email.
modify mkstyle.sh to produce code compatible with our astylerc options.

improve support for testo running from alternate directories.

add option to capture valgrind output in a log file.

fix some url related memory leaks.
add constructor for UrlLink from char*.